of: add optional options parameter to of_find_node_by_path()

Update of_find_node_by_path():
1) Rename function to of_find_node_opts_by_path(), adding an optional
   pointer argument. Provide a static inline wrapper version of
   of_find_node_by_path() which calls the new function with NULL as
   the optional argument.
2) Ignore any part of the path beyond and including the ':' separator.
3) Set the new provided pointer argument to the beginning of the string
   following the ':' separator.
4: Add tests.

Gbp-Pq: Topic features/all
Gbp-Pq: Name of-add-optional-options-parameter-to-of_find_node_by.patch

diff --git a/drivers/of/base.c b/drivers/of/base.c
index 3823edf2d0120d9e89a8195f4845c7d144bcefcf..5910564a390ebe6ab6d64d2e2f25ed4ed2b5129a 100644 (file)
--- a/drivers/of/base.c
+++ b/drivers/of/base.c
@@ -699,10 +699,15 @@ static struct device_node *__of_find_node_by_path(struct device_node *parent,
 {
        struct device_node *child;
        int len = strchrnul(path, '/') - path;
+       int term;
 
        if (!len)
                return NULL;
 
+       term = strchrnul(path, ':') - path;
+       if (term < len)
+               len = term;
+
        __for_each_child_of_node(parent, child) {
                const char *name = strrchr(child->full_name, '/');
                if (WARN(!name, "malformed device_node %s\n", child->full_name))
@@ -715,11 +720,14 @@ static struct device_node *__of_find_node_by_path(struct device_node *parent,
 }
 
 /**
- *     of_find_node_by_path - Find a node matching a full OF path
+ *     of_find_node_opts_by_path - Find a node matching a full OF path
  *     @path: Either the full path to match, or if the path does not
  *            start with '/', the name of a property of the /aliases
  *            node (an alias).  In the case of an alias, the node
  *            matching the alias' value will be returned.
+ *     @opts: Address of a pointer into which to store the start of
+ *            an options string appended to the end of the path with
+ *            a ':' separator.
  *
  *     Valid paths:
  *             /foo/bar        Full path
@@ -729,11 +737,15 @@ static struct device_node *__of_find_node_by_path(struct device_node *parent,
  *     Returns a node pointer with refcount incremented, use
  *     of_node_put() on it when done.
  */
-struct device_node *of_find_node_by_path(const char *path)
+struct device_node *of_find_node_opts_by_path(const char *path, const char **opts)
 {
        struct device_node *np = NULL;
        struct property *pp;
        unsigned long flags;
+       const char *separator = strchr(path, ':');
+
+       if (opts)
+               *opts = separator ? separator + 1 : NULL;
 
        if (strcmp(path, "/") == 0)
                return of_node_get(of_allnodes);
@@ -741,7 +753,7 @@ struct device_node *of_find_node_by_path(const char *path)
        /* The path could begin with an alias */
        if (*path != '/') {
                char *p = strchrnul(path, '/');
-               int len = p - path;
+               int len = separator ? separator - path : p - path;
 
                /* of_aliases must not be NULL */
                if (!of_aliases)
@@ -770,7 +782,7 @@ struct device_node *of_find_node_by_path(const char *path)
        raw_spin_unlock_irqrestore(&devtree_lock, flags);
        return np;
 }
-EXPORT_SYMBOL(of_find_node_by_path);
+EXPORT_SYMBOL(of_find_node_opts_by_path);
 
 /**
  *     of_find_node_by_name - Find a node by its "name" property

diff --git a/include/linux/of.h b/include/linux/of.h
index 29f0adc5f3e47e7c97c4e3aed816437ea9a4fa17..a36be70086ca24697e57ca91e9b75acdbe9746be 100644 (file)
--- a/include/linux/of.h
+++ b/include/linux/of.h
@@ -228,7 +228,13 @@ extern struct device_node *of_find_matching_node_and_match(
        const struct of_device_id *matches,
        const struct of_device_id **match);
 
-extern struct device_node *of_find_node_by_path(const char *path);
+extern struct device_node *of_find_node_opts_by_path(const char *path,
+       const char **opts);
+static inline struct device_node *of_find_node_by_path(const char *path)
+{
+       return of_find_node_opts_by_path(path, NULL);
+}
+
 extern struct device_node *of_find_node_by_phandle(phandle handle);
 extern struct device_node *of_get_parent(const struct device_node *node);
 extern struct device_node *of_get_next_parent(struct device_node *node);
@@ -385,6 +391,12 @@ static inline struct device_node *of_find_node_by_path(const char *path)
        return NULL;
 }
 
+static inline struct device_node *of_find_node_opts_by_path(const char *path,
+       const char **opts)
+{
+       return NULL;
+}
+
 static inline struct device_node *of_get_parent(const struct device_node *node)
 {
        return NULL;